Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Random build #416

Merged
merged 13 commits into from
Aug 28, 2021
Merged

Random build #416

merged 13 commits into from
Aug 28, 2021

Conversation

kerautret
Copy link
Member

PR Description

Following the @dcoeurjo DGtal PR 1588, it implements a random build with also and explicit whithelist. Typical use:
cmake .. -DDGTALTOOLS_RANDOMIZED_BUILD_THRESHOLD=25 -DDGTALTOOLS_RANDOMIZED_BUILD_WHITELIST="$(cat ../cmake/whiteListBuildTools.txt)"

Checklist

  • Doxygen documentation of the code completed (classes, methods, types, members...).
  • Main tool doxygen documentation (following existing documentation of DGtalTools documentation.
  • Check if it follows the tools structure described in CONTRIBUTING.md
  • New entry in the ChangeLog.md added.
  • Update the readme with potentially a screenshot of the tools if it applies.
  • No warning raised in Debug cmake mode (otherwise, Travis C.I. will fail).

@dcoeurjo
Copy link
Member

Why did you use a white list in txt rather than the direct shell script as in DGtal ?
Is the script not working ?
(Just curious)

@kerautret
Copy link
Member Author

Why did you use a white list in txt rather than the direct shell script as in DGtal ?
Is the script not working ?
(Just curious)
In fact, I first add a static list in link to the use through the DGTal Travis build when no tools will be changed. But sure the script perfectly works and also useful for the travis DGtalTools itself. I add it.

@kerautret
Copy link
Member Author

@dcoeurjo I added the script and add the option in travis and appveyor but I miss the change to re activate travis as you done for DGtal, is it in the setting project ? (I have not the hand on it).

@dcoeurjo
Copy link
Member

I don't know exactly why but I don't have build credits anymore on Travis on my account.
That may be the problem

Copy link
Member

@dcoeurjo dcoeurjo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great 👍

@dcoeurjo dcoeurjo merged commit a8e1a1e into DGtal-team:master Aug 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants